Skip to content

Conversation

@infiniteregrets
Copy link
Member

No description provided.

@greptile-apps
Copy link

greptile-apps bot commented Jan 29, 2026

Greptile Overview

Greptile Summary

Updated the release skill documentation to improve the release process workflow. Added a dry-run validation step (cargo publish --dry-run for both s2-cli and s2-lite packages) before merging to catch publishing issues early. Changed merge strategy from --merge to --squash to align with the project's convention of squashing commits (as documented in CLAUDE.md). Added a new failure debugging step with log inspection command to help troubleshoot failed releases.

Key changes:

  • Added step 5: Dry run validation for both packages before merge
  • Changed step 6: Merge strategy from --merge to --squash
  • Added step 8: Failure debugging with error log filtering
  • Renumbered subsequent steps accordingly

Confidence Score: 5/5

  • This PR is safe to merge with no risk - it only updates documentation
  • Score reflects documentation-only changes that improve the release process with better validation, correct merge strategy, and debugging capabilities
  • No files require special attention

Important Files Changed

Filename Overview
.claude/skills/release/SKILL.md Added dry-run validation step, changed merge strategy from --merge to --squash, added failure debugging step, and renumbered subsequent steps

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant Skill as /release skill
    participant GH as GitHub CLI
    participant CI as GitHub Actions
    participant Cargo as cargo publish

    Dev->>Skill: Execute /release
    Skill->>GH: List open release PRs
    GH-->>Skill: Return release PR
    Skill->>GH: Get commits since last tag
    GH-->>Skill: Return commit list
    Skill->>GH: Fetch PR changelog diff
    GH-->>Skill: Return changelog changes
    
    alt Discrepancies found
        Skill->>GH: Trigger release-plz workflow
        GH-->>Skill: Workflow triggered
    end
    
    Note over Skill,Cargo: NEW: Dry run validation
    Skill->>Cargo: cargo publish -p s2-cli --dry-run
    Cargo-->>Skill: Validation result
    Skill->>Cargo: cargo publish -p s2-lite --dry-run
    Cargo-->>Skill: Validation result
    
    alt Changelog correct & dry-run passes
        Note over Skill,GH: CHANGED: --squash instead of --merge
        Skill->>GH: gh pr merge --squash
        GH->>CI: Trigger release workflow
        CI-->>GH: Release started
        Skill->>GH: Monitor release workflow
        GH-->>Skill: Workflow status
    end
    
    alt Release fails
        Note over Skill,GH: NEW: Failure debugging
        Skill->>GH: Get run logs with error filtering
        GH-->>Skill: Error logs
    end
Loading

@infiniteregrets infiniteregrets changed the title chore: update release skill chore: update release skill and release tag Jan 29, 2026
@infiniteregrets infiniteregrets merged commit b47347b into main Jan 29, 2026
9 checks passed
@infiniteregrets infiniteregrets deleted the chore/update-release-skill branch January 29, 2026 04:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant